Skip to content

Keep CI test expectations aligned with current guardrail flows#234

Merged
NagyVikt merged 1 commit intomainfrom
agent/codex/fix-main-ci-failing-tests-2026-04-21-14-58
Apr 21, 2026
Merged

Keep CI test expectations aligned with current guardrail flows#234
NagyVikt merged 1 commit intomainfrom
agent/codex/fix-main-ci-failing-tests-2026-04-21-14-58

Conversation

@NagyVikt
Copy link
Copy Markdown
Collaborator

Automated by scripts/agent-branch-finish.sh (PR flow).

The failing GitHub Actions run was caused by stale test assumptions rather than a CLI regression. The install suite now seeds repo-local git identity before helper commits, the codex-agent no-merge assertion no longer hardcodes the role segment, and the metadata suite accepts the current doctor dry-run variable name.

Constraint: CI runners do not provide global git author identity for nested test repos

Rejected: Change runtime branch naming or doctor output to satisfy stale tests | behavior already matches the intended contract

Confidence: high

Scope-risk: narrow

Directive: Keep tests focused on user-visible behavior instead of local variable names or fixed role prefixes

Tested: npm test

Not-tested: live GitHub Actions rerun after merge
@NagyVikt NagyVikt merged commit bcecd9c into main Apr 21, 2026
@NagyVikt NagyVikt deleted the agent/codex/fix-main-ci-failing-tests-2026-04-21-14-58 branch April 21, 2026 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant